Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed CLI load modal title background color #4200

Merged
merged 1 commit into from
Oct 3, 2024

Conversation

YarosMallorca
Copy link
Contributor

Fixed the Load from File modal background, which in dark mode was white making the text invisible as shown below:

Before

I modified the background and border to use the CSS variables so it properly adapts to light/dark theme like so:
After

Additionally I fixed the border (that had static color as well, thus breaking in light mode) and the modal background (made it surface color) so it matches all the other parts of the UI.

Copy link

netlify bot commented Oct 2, 2024

Deploy Preview for origin-betaflight-app ready!

Name Link
🔨 Latest commit 3eb1ad8
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-app/deploys/66fdbd74b05c500008d35bdd
😎 Deploy Preview https://deploy-preview-4200.dev.app.betaflight.com
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

sonarqubecloud bot commented Oct 2, 2024

@haslinghuis haslinghuis added this to the 11.0 milestone Oct 2, 2024
@haslinghuis haslinghuis merged commit 4ac6f4c into betaflight:master Oct 3, 2024
7 checks passed
@YarosMallorca YarosMallorca deleted the cli-modal-fix branch October 3, 2024 13:12
demvlad pushed a commit to demvlad/betaflight-configurator that referenced this pull request Nov 14, 2024
OGCJM11 pushed a commit to OGCJM11/betaflight-configurator that referenced this pull request Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

3 participants